Skip to content

fix: update analytics script configuration#807

Merged
nehagup merged 1 commit intomainfrom
fix/update-analytics
Mar 27, 2026
Merged

fix: update analytics script configuration#807
nehagup merged 1 commit intomainfrom
fix/update-analytics

Conversation

@slayerjain
Copy link
Copy Markdown
Member

@slayerjain slayerjain commented Mar 26, 2026 •
edited by dhananjay6561
Loading

Summary

  • Add updated analytics script to improve site usage tracking

Test plan

  • Verify docs site builds successfully
  • Verify script loads correctly on docs pages

🤖 Generated with Claude Code

@slayerjain @claude
fix: update analytics script configuration
463f3b1 
Add updated analytics script to improve site usage tracking.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 26, 2026 13:04
@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot bot commented Mar 26, 2026 •
edited
Loading

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Overview

This PR adds a new telemetry analytics script configuration to the Docusaurus config. The change is minimal and follows the existing patterns in the codebase.

Note: The new script uses async: true without defer: true, which differs from other scripts in the file (e.g., feedback.js, clarity.js, apollo-init.js all use both). This may be intentional for telemetry timing purposes, but if consistency is preferred, consider adding defer: true as well.

Files Reviewed (1 file)
  • docusaurus.config.js - 0 issues

Reviewed by claude-4.5-opus-20251124 · 74,609 tokens

Copilot AI reviewed Mar 26, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Docusaurus site configuration to add a new client-side telemetry/analytics script for improved docs usage tracking.

Changes:

  • Add a new remote telemetry script (telemetry.keploy.io) to the scripts array.
  • Configure the telemetry script via data-* attributes (data-endpoint, data-source).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dhananjay6561 dhananjay6561 mentioned this pull request Mar 27, 2026
Merged
@dhananjay6561
Copy link
Copy Markdown
Contributor

dhananjay6561 commented Mar 27, 2026

All the comments have been resolved in PR #808

@nehagup nehagup merged commit c69827d into main Mar 27, 2026
9 of 12 checks passed
@nehagup nehagup deleted the fix/update-analytics branch March 27, 2026 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dhananjay6561 dhananjay6561 dhananjay6561 left review comments

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@slayerjain @dhananjay6561 @nehagup